Skip to content

Workflow for publishing to oidc-sweden github packages#11

Open
danneleaf wants to merge 5 commits intooidc-sweden:mainfrom
danneleaf:main
Open

Workflow for publishing to oidc-sweden github packages#11
danneleaf wants to merge 5 commits intooidc-sweden:mainfrom
danneleaf:main

Conversation

@danneleaf
Copy link

This PR aims to add GitHub Packages publishing workflow while maintaining existing deployment configuration. This allows us to:

Use GitHub Packages for tagged versions
Keep the existing sonatype configuration

The existing distribution management settings for OSSRH remain unchanged. The GitHub Packages workflow is configured to run only on specific tags, ensuring it doesn't interfere with the main release process.

danneleaf and others added 4 commits December 18, 2024 13:34
Signed-off-by: Johannes Dannelov <johannes.dannelov@digg.se>
…x deprecated mvn flag syntax

Signed-off-by: Johannes Dannelov <johannes.dannelov@digg.se>
Signed-off-by: Johannes Dannelov <johannes.dannelov@digg.se>
…nt-path

chore: add deployment to oidc-sweden packages
@janderssonse
Copy link

@danneleaf Kikade snabbt, saknar lite metadata i pom:en om copyright. Kan reviewa denna senare!

@janderssonse
Copy link

janderssonse commented Jan 8, 2025

Starting to code reviewing this

  • I noticed the project is already publishing releases to mvn central - so we dont need to add this..
  • Also heard this is kind of a poc-project, that eventually will be replaced.

So according to this knowledge, if the last part regarding poc is also correct, I think we should not waste any more time than necessary on it, and I suggest, instead of adding more workflows for also building to GH registry ...( and i was planning on adding even more security checks and other fun stuff) etc we instead:

  • Close this pr and don't merge it
  • Publish the latest codebase 2.0.2 for us to mvn central so we can use that (we need that for related wallet libs, mvn central currently only 2.0.0)

@danneleaf @martin-lindstrom @Razumain Above sounds good? Could @martin-lindstrom or @Razumain publish 2.0.2 for us it would be super, gracias.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants